Skip to content

HDDS-5705. [Ozone-Streaming] Change ByteBufStreamOutput to ByteBufferStreamOutput #2603

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
szetszwo merged 8 commits into from
Sep 8, 2021
Merged

HDDS-5705. [Ozone-Streaming] Change ByteBufStreamOutput to ByteBufferStreamOutput #2603

szetszwo merged 8 commits into from
Sep 8, 2021

Conversation

@kaijchen
Copy link
Member

@kaijchen kaijchen commented Sep 1, 2021

What changes were proposed in this pull request?

Renamed the interface ByteBufStreamOutput to ByteBufferStreamOutput.
Removed dependency of netty-buffer which was introduced earlier.

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-5705

How was this patch tested?

The unit test TestBlockDataStreamOutput is changed accordingly.

@kaijchen kaijchen force-pushed the bytebuffer-stream-output branch from c008d45 to 7bb9ddd Compare September 1, 2021 11:59
@captainzmc
Copy link
Member

captainzmc commented Sep 7, 2021

Change looks good. Can you run the CI again? Make sure the CI is correct.

szetszwo
szetszwo approved these changes Sep 8, 2021
View reviewed changes
Copy link
Contributor

@szetszwo szetszwo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 the change looks good.

@szetszwo szetszwo merged commit cd5bfd1 into apache:HDDS-4454 Sep 8, 2021
szetszwo pushed a commit that referenced this pull request Sep 8, 2021
@kaijchen @szetszwo
HDDS-5705. [Ozone-Streaming] Change ByteBufStreamOutput to ByteBuffer…
4156bf7 
…StreamOutput (#2603)
szetszwo pushed a commit that referenced this pull request Oct 1, 2021
@kaijchen @szetszwo
HDDS-5705. [Ozone-Streaming] Change ByteBufStreamOutput to ByteBuffer…
e064c4b 
…StreamOutput (#2603)
sadanand48 pushed a commit that referenced this pull request Oct 2, 2021
@kaijchen
HDDS-5705. [Ozone-Streaming] Change ByteBufStreamOutput to ByteBuffer…
8a090f0 
…StreamOutput (#2603)
szetszwo pushed a commit that referenced this pull request Oct 19, 2021
@kaijchen @szetszwo
HDDS-5705. [Ozone-Streaming] Change ByteBufStreamOutput to ByteBuffer…
14ba242 
…StreamOutput (#2603)
szetszwo pushed a commit that referenced this pull request Oct 28, 2021
@kaijchen @szetszwo
HDDS-5705. [Ozone-Streaming] Change ByteBufStreamOutput to ByteBuffer…
c6950c7 
…StreamOutput (#2603)
szetszwo pushed a commit that referenced this pull request Nov 15, 2021
@kaijchen @szetszwo
HDDS-5705. [Ozone-Streaming] Change ByteBufStreamOutput to ByteBuffer…
d1cb3f9 
…StreamOutput (#2603)
szetszwo pushed a commit that referenced this pull request Nov 30, 2021
@kaijchen @szetszwo
HDDS-5705. [Ozone-Streaming] Change ByteBufStreamOutput to ByteBuffer…
7126b3b 
…StreamOutput (#2603)
captainzmc pushed a commit that referenced this pull request Dec 20, 2021
@kaijchen @captainzmc
HDDS-5705. [Ozone-Streaming] Change ByteBufStreamOutput to ByteBuffer…
fbe9ded 
…StreamOutput (#2603)
szetszwo pushed a commit that referenced this pull request Dec 30, 2021
@kaijchen @szetszwo
HDDS-5705. [Ozone-Streaming] Change ByteBufStreamOutput to ByteBuffer…
435f8da 
…StreamOutput (#2603)
szetszwo pushed a commit that referenced this pull request Jan 20, 2022
@kaijchen @szetszwo
HDDS-5705. [Ozone-Streaming] Change ByteBufStreamOutput to ByteBuffer…
128bccb 
…StreamOutput (#2603)
captainzmc pushed a commit that referenced this pull request Feb 9, 2022
@kaijchen @captainzmc
HDDS-5705. [Ozone-Streaming] Change ByteBufStreamOutput to ByteBuffer…
8e30bfd 
…StreamOutput (#2603)
szetszwo pushed a commit that referenced this pull request Feb 16, 2022
@kaijchen @szetszwo
HDDS-5705. [Ozone-Streaming] Change ByteBufStreamOutput to ByteBuffer…
7b2a704 
…StreamOutput (#2603)
@kaijchen kaijchen deleted the bytebuffer-stream-output branch February 25, 2022 02:49
szetszwo pushed a commit that referenced this pull request Mar 15, 2022
@kaijchen @szetszwo
HDDS-5705. [Ozone-Streaming] Change ByteBufStreamOutput to ByteBuffer…
50e96dd 
…StreamOutput (#2603)
szetszwo pushed a commit that referenced this pull request Mar 24, 2022
@kaijchen @szetszwo
HDDS-5705. [Ozone-Streaming] Change ByteBufStreamOutput to ByteBuffer…
4464d8a 
…StreamOutput (#2603)
szetszwo pushed a commit to szetszwo/ozone that referenced this pull request May 6, 2022
@kaijchen @szetszwo
HDDS-5705. [Ozone-Streaming] Change ByteBufStreamOutput to ByteBuffer…
3479f54 
…StreamOutput (apache#2603)
szetszwo pushed a commit that referenced this pull request May 13, 2022
@kaijchen @szetszwo
HDDS-5705. [Ozone-Streaming] Change ByteBufStreamOutput to ByteBuffer…
6c1f954 
…StreamOutput (#2603)
szetszwo pushed a commit that referenced this pull request May 24, 2022
@kaijchen @szetszwo
HDDS-5705. [Ozone-Streaming] Change ByteBufStreamOutput to ByteBuffer…
05b1910 
…StreamOutput (#2603)
szetszwo pushed a commit that referenced this pull request Jun 9, 2022
@kaijchen @szetszwo
HDDS-5705. [Ozone-Streaming] Change ByteBufStreamOutput to ByteBuffer…
f7e5990 
…StreamOutput (#2603)
captainzmc pushed a commit to captainzmc/hadoop-ozone that referenced this pull request Jul 4, 2022
@kaijchen @captainzmc
HDDS-5705. [Ozone-Streaming] Change ByteBufStreamOutput to ByteBuffer…
3ea9af3 
…StreamOutput (apache#2603)
szetszwo pushed a commit that referenced this pull request Oct 25, 2022
@kaijchen @szetszwo
HDDS-5705. [Ozone-Streaming] Change ByteBufStreamOutput to ByteBuffer…
81b9f3d 
…StreamOutput (#2603)

(cherry picked from commit 3ea9af3)
(cherry picked from commit 74c5f0f5fca4475bb4369fd89ab07424fa87008b)
szetszwo pushed a commit that referenced this pull request Nov 7, 2022
@kaijchen @szetszwo
HDDS-5705. [Ozone-Streaming] Change ByteBufStreamOutput to ByteBuffer…
8853c8a 
…StreamOutput (#2603)

(cherry picked from commit 3ea9af3)
(cherry picked from commit 74c5f0f5fca4475bb4369fd89ab07424fa87008b)
(cherry picked from commit 81b9f3d)
szetszwo pushed a commit that referenced this pull request Dec 1, 2022
@kaijchen @szetszwo
HDDS-5705. [Ozone-Streaming] Change ByteBufStreamOutput to ByteBuffer…
ae02e96 
…StreamOutput (#2603)

(cherry picked from commit 8853c8a)
szetszwo pushed a commit that referenced this pull request Dec 16, 2022
@kaijchen @szetszwo
HDDS-5705. [Ozone-Streaming] Change ByteBufStreamOutput to ByteBuffer…
ff59eeb 
…StreamOutput (#2603)

(cherry picked from commit 8853c8a)
nishitpatira pushed a commit to nishitpatira/ozone that referenced this pull request Dec 16, 2022
@kaijchen
HDDS-5705. [Ozone-Streaming] Change ByteBufStreamOutput to ByteBuffer…
1283af9 
…StreamOutput (apache#2603)

(cherry picked from commit 8853c8a)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@szetszwo szetszwo szetszwo approved these changes

@captainzmc captainzmc Awaiting requested review from captainzmc

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@kaijchen @captainzmc @szetszwo